Skip to content

feat: [STO-12076]: add sast_remediation_diff MCP resource - #701

Merged
thisrohangupta merged 4 commits into
harness:mainfrom
priyanshu11137:feat/STO-12076/sast-remediation-diff
Jul 29, 2026
Merged

feat: [STO-12076]: add sast_remediation_diff MCP resource#701
thisrohangupta merged 4 commits into
harness:mainfrom
priyanshu11137:feat/STO-12076/sast-remediation-diff

Conversation

@priyanshu11137

@priyanshu11137 priyanshu11137 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Expose STO DiffOccurrences so agents can list existing vs new occurrences after a validation execution.

Description

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other

Checklist

  • pnpm test passes
  • pnpm typecheck passes
  • pnpm build passes
  • pnpm standards:check passes (architecture guardrails — see docs/coding-standards.md)
  • pnpm docs:check passes (if registry/tool counts changed)

Coding Standards (registry-driven MCP model)

If this PR adds or changes Harness API coverage:

  • No new server.registerTool() calls — only toolset definitions in src/registry/toolsets/
  • Toolset registered in ALL_TOOLSETS and ToolsetName union
  • operationPolicy on every new/changed endpoint
  • Shared response extractors from src/registry/extractors.ts (no raw passthrough on real endpoints)
  • identifierFields and scope declared on new resources
  • No console.log() in src/ (stdio JSON-RPC safety)

Expose STO DiffOccurrences so agents can list existing vs new
occurrences after a validation execution.

Co-authored-by: Cursor <cursoragent@cursor.com>
@CLAassistant

CLAassistant commented Jul 27, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ thisrohangupta
❌ priyanshujindal-harness
You have signed the CLA already but the status is still pending? Let us recheck it.

Architecture guardrail caps inline responseExtractor count in sto.ts.

Co-authored-by: Cursor <cursoragent@cursor.com>
@priyanshu11137
priyanshu11137 force-pushed the feat/STO-12076/sast-remediation-diff branch from 041a5f2 to ff5e554 Compare July 27, 2026 12:27

@thisrohangupta thisrohangupta left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adds sast_remediation_diff as a declarative STO toolset resource (no new tool handler), with a shared extractor, operationPolicy, preflight validation, and 26 tests. Resolved the README conflict with #707 (kept badge + bumped count to 216). Verified locally: build, docs:check, standards (80/80), STO tests (26/26) all pass.

@thisrohangupta
thisrohangupta merged commit 508d6d6 into harness:main Jul 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants